Skip to content

Conversation

@jfhamlin
Copy link
Contributor

@jfhamlin jfhamlin commented Sep 6, 2025

  • Add explicit loading of namespaces required by AOT-compiled core (protocols, string, go.io)
    • Fixes UnboundVar errors caused by missing transitive dependencies
    • TODO: AOT should automatically identify and load these dependencies
  • Fix dynamic var generation to capture root binding using fresh goroutine
    • Ensures vars are initialized with root values, not compile-time dynamic bindings
  • Add AOT compilation for additional stdlib namespaces (go.io, go.types, uuid)
  • Set GLOJURE_USE_AOT=false during AOT generation to avoid circular dependencies
  • Skip non-var namespace mappings during codegen instead of panicking
  • Clean up leftover generated loader files (core_deftype, core_print) from AOT development
  • Consolidate AOT loader imports in pkg/glj/init.go

These changes resolve runtime errors when using AOT-compiled namespaces by ensuring all required dependencies are loaded.

- Add explicit loading of namespaces required by AOT-compiled core (protocols, string, go.io)
  - Fixes UnboundVar errors caused by missing transitive dependencies
  - TODO: AOT should automatically identify and load these dependencies
- Fix dynamic var generation to capture root binding using fresh goroutine
  - Ensures vars are initialized with root values, not compile-time dynamic bindings
- Add AOT compilation for additional stdlib namespaces (go.io, go.types, uuid)
- Set GLOJURE_USE_AOT=false during AOT generation to avoid circular dependencies
- Skip non-var namespace mappings during codegen instead of panicking
- Clean up leftover generated loader files (core_deftype, core_print) from AOT development
- Consolidate AOT loader imports in pkg/glj/init.go

These changes resolve runtime errors when using AOT-compiled namespaces by ensuring
all required dependencies are loaded.
@jfhamlin jfhamlin merged commit c48eec5 into main Sep 6, 2025
2 checks passed
@jfhamlin jfhamlin deleted the fix/unbound branch September 6, 2025 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants